Skip to content

Conversation

@odeimaiz
Copy link
Member

@odeimaiz odeimaiz commented Apr 14, 2025

What do these changes do?

  • Make username's font smaller and move it to a header
  • Make timestamp's font smaller and move it to a header
  • Add a bubbly background to the message

Bonus:

  • Bugfix: share with groupId
  • Bugfix: Add recipient to Note

Before:
image

After:
CommentsUI

Related issue/s

How to test

Dev-ops checklist

@odeimaiz odeimaiz self-assigned this Apr 14, 2025
@odeimaiz odeimaiz added this to the Pauwel Kwak milestone Apr 14, 2025
@odeimaiz odeimaiz added t:enhancement Improvement or request on an existing feature a:frontend issue affecting the front-end (area group) bug buggy, it does not work as expected labels Apr 14, 2025
@odeimaiz odeimaiz changed the title 🐛🎨 [Frontend] Enh: Comment UI 🐛🎨 [Frontend] Enh: Comments aesthetics Apr 14, 2025
@odeimaiz odeimaiz marked this pull request as ready for review April 14, 2025 12:24
@sonarqubecloud
Copy link

@odeimaiz odeimaiz added the 🤖-automerge marks PR as ready to be merged for Mergify label Apr 14, 2025
@odeimaiz
Copy link
Member Author

@Mergifyio queue

@mergify
Copy link
Contributor

mergify bot commented Apr 14, 2025

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at 69c60ad

@odeimaiz odeimaiz requested a review from bisgaard-itis April 14, 2025 13:28
@mergify mergify bot merged commit 69c60ad into ITISFoundation:master Apr 14, 2025
58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🤖-automerge marks PR as ready to be merged for Mergify a:frontend issue affecting the front-end (area group) bug buggy, it does not work as expected t:enhancement Improvement or request on an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants